Skip to content

Added support for multi root workspaces with the new language server server - #4244

Merged
Kartik Raj (karrtikr) merged 12 commits into
microsoft:masterfrom
karrtikr:lsSupport
Feb 12, 2019
Merged

Added support for multi root workspaces with the new language server server#4244
Kartik Raj (karrtikr) merged 12 commits into
microsoft:masterfrom
karrtikr:lsSupport

Conversation

@karrtikr

@karrtikr Kartik Raj (karrtikr) commented Jan 31, 2019

Copy link
Copy Markdown

For #3008

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Has a news entry file (remember to thank yourself!)
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated
  • Test plan is updated as appropriate
  • package-lock.json has been regenerated by running npm install (if dependencies have changed)

@codecov

codecov Bot commented Feb 1, 2019

Copy link
Copy Markdown

Codecov Report

Merging #4244 into master will increase coverage by 22%.
The diff coverage is 74%.

@@           Coverage Diff            @@
##           master   #4244     +/-   ##
========================================
+ Coverage      56%     78%    +22%     
========================================
  Files         361     438     +77     
  Lines       15274   20390   +5116     
  Branches     2373    3273    +900     
========================================
+ Hits         8494   15721   +7227     
+ Misses       6236    4665   -1571     
+ Partials      544       4    -540
Flag Coverage Δ
#Linux 66% <50%> (?)
#Windows 67% <50%> (?)
#macOS 66% <50%> (?)

@karrtikr
Kartik Raj (karrtikr) force-pushed the lsSupport branch 4 times, most recently from c1cc3de to 1e77898 Compare February 1, 2019 00:57
@karrtikr Kartik Raj (karrtikr) changed the title Added support for multi root workspaces with the new language server server WIP - Added support for multi root workspaces with the new language server server Feb 5, 2019
@karrtikr
Kartik Raj (karrtikr) force-pushed the lsSupport branch 11 times, most recently from 6a44a0b to 16f6d20 Compare February 7, 2019 05:08
@karrtikr Kartik Raj (karrtikr) changed the title WIP - Added support for multi root workspaces with the new language server server Added support for multi root workspaces with the new language server server Feb 12, 2019
@karrtikr Kartik Raj (karrtikr) changed the title Added support for multi root workspaces with the new language server server WIP - Added support for multi root workspaces with the new language server server Feb 12, 2019
@karrtikr Kartik Raj (karrtikr) changed the title WIP - Added support for multi root workspaces with the new language server server Added support for multi root workspaces with the new language server server Feb 12, 2019
@karrtikr Kartik Raj (karrtikr) changed the title Added support for multi root workspaces with the new language server server WIP - Added support for multi root workspaces with the new language server server Feb 12, 2019
@karrtikr Kartik Raj (karrtikr) changed the title WIP - Added support for multi root workspaces with the new language server server Added support for multi root workspaces with the new language server server Feb 12, 2019
Comment thread src/client/activation/languageServer/manager.ts Outdated
Comment thread src/client/activation/languageServer/manager.ts Outdated
Comment thread src/client/activation/activationService.ts Outdated
@karrtikr
Kartik Raj (karrtikr) merged commit cf686d2 into microsoft:master Feb 12, 2019
@karrtikr
Kartik Raj (karrtikr) deleted the lsSupport branch February 12, 2019 22:27
@lock lock Bot locked as resolved and limited conversation to collaborators Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants